Skip to content

Enabled test_pthread_dylink_tls under STACK_OVERFLOW_CHECK=2#26890

Merged
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:test_pthread_dylink_tls
May 8, 2026
Merged

Enabled test_pthread_dylink_tls under STACK_OVERFLOW_CHECK=2#26890
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:test_pthread_dylink_tls

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented May 8, 2026

The actual fix for this was combination of #26878 and WebAssembly/binaryen#8679

Fixes: #24964 Please enter the commit message for your changes. Lines starting

@sbc100 sbc100 requested review from juj and kripken May 8, 2026 00:07
@sbc100 sbc100 changed the title Enabled test_pthread_dylink_tls under STACK_OVERFLOW_CHECK=2 Enabled test_pthread_dylink_tls under STACK_OVERFLOW_CHECK=2 May 8, 2026
The actual fix for this was combination of emscripten-core#26878 and
WebAssembly/binaryen#8679

Fixes: emscripten-core#24964 Please enter the commit message for your changes. Lines starting
@sbc100 sbc100 force-pushed the test_pthread_dylink_tls branch from 1ef0fe9 to 7069c59 Compare May 8, 2026 06:10
@sbc100 sbc100 enabled auto-merge (squash) May 8, 2026 07:43
@sbc100 sbc100 merged commit 0d8be82 into emscripten-core:main May 8, 2026
30 checks passed
@sbc100 sbc100 deleted the test_pthread_dylink_tls branch May 8, 2026 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stack overflow check in pthread+dynamic linking mode test failure

2 participants